docs(skill): verified anti-pattern deltas for formula + data authoring skills#1922
Merged
Merged
Conversation
…g skills Continues the upstream "teach the AI the right pattern" work for the other two authoring skills. Both were already strong (formula already documents the #1491 brace-in-CEL mistake + dialects + isBlank; data already warns about inverted script validation), so this adds ONLY verified deltas from this season — no fabricated/padded anti-patterns: - objectstack-formula: only the stdlib + CEL built-ins are callable; an unknown function (PRIOR(), legacy ISBLANK(), a typo) now FAILS objectstack build ("no matching overload"), not a silent runtime no-op (#1877). - objectstack-data: `multiple: true` lookup is an ARRAY column ≠ junction object — reference positionally `{record.tags.0}` (#1872); on insert an omitted optional field reads as null in validation predicates (#1871). Docs-only; check:skill-docs passes (frontmatter-derived docs unaffected); empty changeset. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
github-actions
Bot
added
documentation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Continues the upstream "teach the AI the right pattern at the source" work (after #1921 for the automation skill) to the other two authoring skills.
Discipline: verified deltas only, no padding
Both skills are already mature — formula already documents the #1491 brace-in-CEL mistake + the dialect split +
isBlank/null handling; data already warns that script validation is inverted. So this adds only the verified gaps from this session, not fabricated anti-patterns:objectstack-formula — only the stdlib + CEL built-ins are callable; an UNKNOWN function (
PRIOR(), legacyISBLANK(), a typo'disBlnk()) now failsobjectstack buildwith a "no matching overload" type error, not a silent runtime no-op. Useprevious.x/isBlank(). (#1877)objectstack-data
multiple: truelookup is an array column, not a junction object — reference positionally ({record.tags.0}); use a junction object only when the relationship carries attributes. (This is the exact confusion in [P2] record-change context does not hydrate multi-lookup (junction) fields #1872.)nullin a validation predicate, sorecord.due_date == nullmatches an omitted field the same as explicitnull. ([P2] Script validationfield == nulldoes not match a field omitted on insert #1871)Verification
Docs-only.
pnpm check:skill-docspasses (generated docs derive from SKILL.md frontmatter, unchanged). Empty changeset.🤖 Generated with Claude Code